fix: resolve cross-file consistency issues and doc gaps#5
Merged
Conversation
- standardize the chapters artifact on chapters.md everywhere; the post-production workflow wrote chapters.txt while the YouTube publish workflow looked for chapters.md - point the launcher's "pi is not installed" error at npm install and the renamed @earendil-works/pi-coding-agent package - reduce /phase1 to a thin alias for /full-review and drop it from the startup widget (also dedupe /cut-clips) - give the "no transcription backend found" error actionable uv sync --group commands - document the OAuth test-user step in the README, testing-mode refresh token expiry in the publisher skill, uv first-run venv behavior, and the non-macOS scanner skip rule (AGENTS.md + skill) - move /login guidance to after first launch in Quick start; note that Common workflows commands are reference, not required UX Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
First of a series of PRs from a whole-project review (architecture / code quality / DX passes). This one is the consistency and documentation batch:
chapters.md— the post-production skill and/chapters//full-reviewprompts told pi to writechapters.txt, while the YouTube publisher skill,/publish-youtube, and README looked forchapters.md. The chapters → publish handoff referenced a file the upstream step never wrote.npm install -g @mariozechner/pi-coding-agent(renamed in chore: bump pi to @earendil-works/pi-coding-agent ^0.78.1 #3). Now points atnpm installwith the@earendil-workspackage as the global alternative./phase1demoted — its prompt file already said "prefer /full-review" but the startup widget advertised it first; now a 1-line alias, removed from the widget (which also listed/cut-clipstwice).uv sync --group transcribe-*/uv run --group ...commands instead of generic package names.access_denied), testing-mode 7-day refresh-token expiry, uv first-run.venvcreation, Linux install note, explicit "skip the scanner on non-macOS" rule for pi,/loginguidance moved to after first launch,preferred_reviewspelling matched to the example toml.Test plan
format:check/lint/typecheck/npm test(7/7) pass locally🤖 Generated with Claude Code